Greptile Overview

Summary

The key changes involve two files:

1. next.config.ts: The rewrites() function now unconditionally returns PostHog proxy routes (/ingest/static/:path* and /ingest/:path*) that forward requests to PostHog's US assets and API endpoints. Previously, these routes were only added when POSTHOG_ENABLED was truthy.
2. lib/env.ts: Removes the unused server-side POSTHOG_ENABLED environment variable from the Zod schema validation, while keeping the client-side PostHog variables (NEXT_PUBLIC_POSTHOG_ENABLED and NEXT_PUBLIC_POSTHOG_KEY) intact.

This approach aligns with best practices for analytics integration where reverse proxy routes should always be available to avoid CORS issues and ad blocker interference. PostHog functionality is now controlled entirely through client-side environment variables, which is more appropriate since PostHog is primarily a client-side analytics service. The reverse proxy setup ensures reliable communication with PostHog's servers by routing requests through the application's domain rather than making direct cross-origin requests.

Important Files Changed

Confidence score: 5/5

• This PR is safe to merge with minimal risk as it simplifies configuration and removes unused code
• Score reflects straightforward changes that improve reliability by removing conditional complexity
• No files require special attention as both changes are clean removals of unnecessary conditional logic

Sequence Diagram

sequenceDiagram
    participant User
    participant App as "Next.js App"
    participant PostHog as "PostHog Analytics"
    
    User->>App: "Visit app page"
    App->>App: "Load Next.js config"
    App->>App: "Apply rewrites unconditionally"
    Note over App: "Rewrites now added without POSTHOG_ENABLED check"
    
    User->>App: "Request /ingest/static/:path*"
    App->>PostHog: "Proxy to https://us-assets.i.posthog.com/static/:path*"
    PostHog->>App: "Return static assets"
    App->>User: "Return proxied assets"
    
    User->>App: "Request /ingest/:path*"
    App->>PostHog: "Proxy to https://us.i.posthog.com/:path*"
    PostHog->>App: "Return analytics data"
    App->>User: "Return proxied response"
    
    Note over App: "POSTHOG_ENABLED env var removed from env.ts"
    Note over App: "Rewrites always active for reverse proxy"

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}